Skip to content

fix: avoid reextracting superseded raw items - #98

Merged
Teingi merged 1 commit into
ob-labs:mainfrom
zshs000:fix/issue90-raw-idempotency
Jul 31, 2026
Merged

fix: avoid reextracting superseded raw items#98
Teingi merged 1 commit into
ob-labs:mainfrom
zshs000:fix/issue90-raw-idempotency

Conversation

@zshs000

@zshs000 zshs000 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix #90.

Prevent the same raw item from being extracted repeatedly when compact() is called multiple times. After the first
extraction, the raw item has superseded_by set, so subsequent compaction cycles must skip it.

Solution Description

  • Added not it.superseded_by to the raw -> extracted candidate filter.
  • Added a regression test covering repeated evolution of the same superseded raw item.
  • Verified the test failed before the fix and passed after the fix.
  • All evolution tests pass: 48 passed.

This prevents duplicate extracted items from being generated from the same raw item.

@Teingi Teingi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Teingi
Teingi merged commit c386eac into ob-labs:main Jul 31, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: 同一1个raw执行了2次compact,会生成2个extracted

2 participants